[BUG] KUKA KRC5#1151
Merged
Merged
Conversation
Collaborator
Author
|
Freely configurable user byte array added. |
Brano5
approved these changes
May 21, 2026
…ate-mirror diagnostics - Introduced `DataFromPlcToRobot` and `DataFromRobotToPlc` for application-defined data exchange. - Added per-axis coordinate-mirror task-`potential` identifiers for improved diagnostics. - Updated safety message handling to reflect informational status on KRC5. - Revised documentation to clarify API divergence from AxoKrc4 and added new data exchange section.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
AxoKrc5gains raw application-defined passthrough members (DataFromPlcToRobot/DataFromRobotToPlc), per-axis coordinate-mirror diagnostics (task-potentialIDs1501–1506/1511–1516), and message20002(auto-mode lost mid-task) is downgradedError→Info.AxoKrc5.md,TROUBLES.md, per-library + rootCHANGELOG.md) updated to document the new surface and behaviour.AxoKrc4is intentionally left unchanged — KRC4 and KRC5 are no longer API/behaviour-identical, and the docs now state that divergence explicitly.Changes from CHANGELOG
Workspace
[KUKA] KRC5 raw data exchange, coordinate-mirror diagnostics, and auto-mode severity fix
Note: KRC5-only change in
src/components.kuka.robotics/.AxoKrc5now diverges fromAxoKrc4in three respects —AxoKrc4is intentionally left unchanged in this PR. Fixes #1148.AxoKrc5exposes raw application-defined passthrough membersDataFromPlcToRobot : ARRAY[0..19] OF BYTE(PLC → robot, output bytes_data[44..63]) andDataFromRobotToPlc : ARRAY[0..15] OF BYTE(robot → PLC, input bytes_data[48..63]). Both carryRenderIgnore;Run()transports them verbatim without interpretation. Wrapped in the new<AxoKrc5DataExchangeDeclaration>source region.AxoKrc5adds per-axis coordinate-mirror task-potentialidentifiers1501–1506(StartMotorsProgramAndMovements) and1511–1516(StartMovements), raised viaTaskMessengerwhile waiting for eachInputs.Coordinates.{X,Y,Z,Rx,Ry,Rz}to mirror the commanded value within0.01tolerance. Matching.NETtwin entries added to both theTaskMessengertext list anderrorDescriptionDictinAxoKrc5.cs.AxoKrc5safety message20002(Inputs.Automatic = FALSEwhile a task is busy) is now raised as categoryInfoinstead ofError— losing auto mode mid-task is informational on KRC5, not a hard fault.AxoKrc5_v_5_x_x_Showcase.stgained an "Exchange raw data with the robot" sequencer step (in//<DataExchange>markers);Stepswidened[0..19]→[0..20],_lastByteFromRobotstatus field added.AxoKrc5.mdrelaxed the "identical public API toAxoKrc4" wording, added a Data exchange section (library declaration + showcase usage refs), and a note listing the three KRC5-only differences.TROUBLES.mdflags the per-class differences (1501–1516, 20002 severity split). Appended0.54.0entry tosrc/components.kuka.robotics/docs/CHANGELOG.md.Impact:
potentialIDs) rather than a single "coordinates not mirrored" wait.Risks/Review:
AxoKrc4andAxoKrc5are no longer API/behaviour-identical. Documentation now states the divergence explicitly; if a follow-up backports these changes toAxoKrc4, the "KRC5-only" wording inAxoKrc5.md/TROUBLES.mdmust be reverted.44..63) and robot→PLC (48..63) windows overlap on the same physical I/O block in opposite directions — verify the controller-side mapping matches before relying on the passthrough.Testing:
apax ibinsrc/showcase/app/— the new KRC5 data-exchange sequencer step builds and runs through toCompleteSequence.dotnet buildonsrc/showcase/app/ix-blazor/showcase.blazor/— theAxoKrc5.csmessenger/error-dictionary additions compile.scripts/_build_documentation.ps1— the new[!code-pascal[]](showcaseDataExchange) and[!code-smalltalk[]](AxoKrc5DataExchangeDeclaration) directives resolve.Commits
Other
8d36c5b68Create draft PR for [BUG] missing alarms when coordinate mismatch, superfluous alarm not-in-auto, user-defined bytes #11487b2cdab7bwipDiff stat
Test plan
apax build; affected sequences exercised in PLCSIM AdvancedGenerated by
/pr-description-update. Last regenerated: 2026-05-21.